Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add log flume lifthill sound #19

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

karst
Copy link
Member

@karst karst commented Feb 15, 2024

I got it on a nice volume level and looping correctly. I like the overall feel it gives to the log flume and makes it more of a present ride. This was supposed to go to OpenRCT2#20258, but I might decide to make a seperate PR for it.

Requires the updating of audio.h on the OpenRCT2 main repo.

In my test PR I called this LiftFlume (Row 133): https://github.com/karst/OpenRCT2/tree/log-flume-sound

I got it on a nice volume level and looping correctly. I like the overall feel it gives to the log flume and makes it more of a present ride.
This was supposed to go to OpenRCT2#20258, but I might decide to make a seperate PR for it.
Copy link
Member

@tupaschoal tupaschoal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it, always felt weird that log flumes would be completely silent uphill

I got it on a nice volume level and looping correctly.
Was this extracted from somewhere? I was a bit confused

@karst
Copy link
Member Author

karst commented Feb 15, 2024

i had it ready a few months ago already, but never published it. It wasn't looping correctly and it was a tad loud.

That makes me think i should probably go over all sfx again sometime to adjust them, especially the toilet sound apparently that's way too loud for people 😂

Copy link
Member

@tupaschoal tupaschoal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it deserves its own PR on OpenRCT2/OpenRCT2, it's a simpler change, so it doesn't get dragged with the other PR.

@karst
Copy link
Member Author

karst commented Feb 15, 2024

The only issue will be that it will only be audible on currently cheated in lifthills. Ofcourse the other PR will take care of that.

@karst karst merged commit 7ee7648 into OpenRCT2:master Feb 15, 2024
1 check passed
@karst karst deleted the log_flume_lift branch February 15, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants